-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Layout::Flex
- Component implementation
#2751
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
363c966
to
240d50c
Compare
Flex
- Component implementationLayout::Flex
- Component implementation
24bd31c
to
757e032
Compare
c28eca6
to
566566c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking really great, and covers a lot of use cases. I added a couple suggestions that would cover some additional flex box features. If we are trying to keep the component less complex for consumers then may not need the features I'm suggesting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is looking quite good overall. What do you think about adding @width
and @height
options though?
Or perhaps those could be considered together with break points & responsiveness in a phase 2 release? (I guess we already discussed doing these as a follow-on, sorry.)
Layout::Flex
- Component implementationLayout::Flex
- Component implementation
📌 Summary
This PR implements the
Flex
component (HDS-4531) in the context of the wider initiative for spacing/layout/pages components/helpers (HDS-4527).There is an equivalent PR to implement a grid component: #2756
🛠️ Detailed description
In this PR I have:
Hds::Layout::Flex
componentFlex
The component documentation is done in #2764
👉 👉 👉 Preview: https://hds-showcase-git-flex-component-implementation-hashicorp.vercel.app/layouts/flex
🔗 External links
Jira ticket: https://hashicorp.atlassian.net/browse/HDS-4635
Project plan: HDS Project Plan - Flex/Grid components
👀 Component checklist
💬 Please consider using conventional comments when reviewing this PR.